Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Learning P4 Page #25

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Learning P4 Page #25

wants to merge 1 commit into from

Conversation

sibanez12
Copy link
Contributor

Added initial draft of a "Learning P4" page.

A couple of notes:

  • Currently, the videos are uploaded to my personal account. Not sure if it would be better to upload them to the P4 youtube channel and link to those instead.

  • We should probably upload them to youku as well and show those instead if the client is in China.

  • I removed the FAQ section that I had initially because I saw that there is already an FAQ page. I'd like to add a link to that page once it is available.

  • I'd also like to add a link to the slides, but I can't find them on Google Drive anymore. Have they been moved since the P4 Developer Day?

Copy link
Collaborator

@jnfoster jnfoster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Steve,

We'll want to add much more to this, but for an initial version, it looks good.

Should we merge?

-N

@sibanez12
Copy link
Contributor Author

I agree, this page can become much more detailed and comprehensive. I wasn't sure if it was a good idea to replicate much of the information that is already contained in the README files for the exercises. If we do that then it will require extra effort to maintain consistency between the website and the README files should we ever decide to update the exercises. Any thoughts here?

I'm ok with merging this as an initial version.

-Steve

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants